Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(DOCSP-34109) Trigger Throughput Wording #627

Merged
merged 2 commits into from
Nov 7, 2023
Merged

Conversation

nlarew
Copy link
Contributor

@nlarew nlarew commented Nov 6, 2023

Pull Request Info

Jira

Staged Changes

Copy link

@tsedgwick tsedgwick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for updating this!

quick drive by - this was flagged to me.

filter. For the best performance, we recommend limiting the size of
each change event to 2KB or less.
filter. For the best performance, limit the size of each change event
to 2KB or less.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nlarew does it make sense to link to the $projection section of db triggers for this? or do you think that's discoverable enough when you are on this page?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Couldn't hurt to add a link!


If the linked data source is a dedicated server (M10+ Tier),
you can configure individual triggers to process up to 50,000 events concurrently.
Trigger capacity is not a direct measure of throughput or a guaranteed

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏🏻

@dpish
Copy link

dpish commented Nov 7, 2023

LGTM

Copy link

@laurazhukas laurazhukas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀

Copy link
Collaborator

@krollins-mdb krollins-mdb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks for doing this, @nlarew!

@nlarew nlarew merged commit 443c845 into mongodb:master Nov 7, 2023
4 checks passed
@nlarew nlarew deleted the throughput branch November 7, 2023 22:06
@docs-builder-bot
Copy link

MongoCaleb pushed a commit to MongoCaleb/docs-app-services that referenced this pull request Jun 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants